Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gazebo_ros_openni_kinect plugin: adds publishing of the camera info again (fixes #95) #97

Merged
merged 1 commit into from
Aug 9, 2013

Conversation

bit-pirate
Copy link
Contributor

No description provided.

@davetcoleman
Copy link
Collaborator

Related to #95 (comment)

@piyushk
Copy link
Contributor

piyushk commented Aug 9, 2013

I've tested this and it works fine.

davetcoleman added a commit that referenced this pull request Aug 9, 2013
gazebo_ros_openni_kinect plugin: adds publishing of the camera info again (fixes #95)
@davetcoleman davetcoleman merged commit 5fac6e0 into ros-simulation:hydro-devel Aug 9, 2013
@davetcoleman
Copy link
Collaborator

Thanks!

@piyushk
Copy link
Contributor

piyushk commented Aug 21, 2013

@davetcoleman @bit-pirate

I am still seeing sporadic instances where gazebo_ros_openni_kinect does not publish depth information. Are either of you seeing something like this?

@bit-pirate
Copy link
Contributor Author

I haven't seen any glitches so far, but I also haven't been testing the turtlebot simulation extensively yet.
Does the depth camera info stream cause troubles or the depth stream itself?

@piyushk
Copy link
Contributor

piyushk commented Aug 21, 2013

Both of them are missing. I can't reproduce the error right now, but I get
a synchronization error on repeat from image_transport stating that only 1
CameraInfo message received in the last 10 seconds and 0 Image messages.
However, rostopic echo produces nothing for both. The image stream and
camera info is fine.

On Wed, Aug 21, 2013 at 6:10 PM, Marcus Liebhardt
notifications@git.luolix.topwrote:

I haven't seen any glitches so far, but I also haven't been testing the
turtlebot simulation extensively yet.
Does the depth camera info stream cause troubles or the depth stream
itself?


Reply to this email directly or view it on GitHubhttps://github.com//pull/97#issuecomment-23057606
.

@davetcoleman
Copy link
Collaborator

I'm afraid I haven't been using the openni_kinect plugin and do not have a test setup for this. Too many other projects on my plate...

@bit-pirate
Copy link
Contributor Author

I'll try to test this as soon as possible, but I'm buried in other
things, too.

2013/8/22 Dave Coleman notifications@github.com

I'm afraid I haven't been using the openni_kinect plugin and do not have a
test setup for this. Too many other projects on my plate...

Reply to this email directly or view it on GitHubhttps://github.com//pull/97#issuecomment-23061755
.

Marcus Liebhardt
Adresse: Zimmer 403, 634-1 Seongsan-dong, Mapo-gu, 121-846, Seoul, Republik
Korea
Address: Room 403, 634-1 Seongsan-dong, Mapo-gu, 121-846, Seoul, Republic
of Korea
주소: 대한민국 서울특별시 121-846 마포구 634-1 성산동 4 층 403 호
Mobile / Mobilfunk: +82 10 3322 0566
Email: ich@marcusliebhardt.de ich@marcusliebhardt.de

@piyushk
Copy link
Contributor

piyushk commented Aug 22, 2013

No worries. I don't think it's a crucial issue. I'll get back to it as time permits. In any case, I am unable to use gazebo_ros_openni_kinect in multi-robot experiments because of #99.

@bit-pirate If you get some time, could you take a look at ros-drivers/openni_launch#5? I would like to solve openni_launch issues before beta finishes.

Thanks!!

cohen39 pushed a commit to cohen39/gazebo_ros_pkgs that referenced this pull request Nov 15, 2021
gazebo_ros_openni_kinect plugin: adds publishing of the camera info again (fixes ros-simulation#95)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants